Skip to content

Conversation

ivanpanch
Copy link
Contributor

I have corrected some minor mistakes.

@barendgehrels
Copy link
Collaborator

Thanks a lot - looks very good - I'm basically OK (didn't run the tools myself)

Copy link
Collaborator

@tinko92 tinko92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No objections to the changes. Should the commits be squashed into one, though?

@ivanpanch
Copy link
Contributor Author

I have made some further changes, can you have a look? One last thing: There is Line height in SVG pixels, in case the text contains \n, but \n is not rendered verbatim here: https://www.boost.org/doc/libs/latest/libs/geometry/doc/html/geometry/reference/io/svg/svg_mapper.html

@barendgehrels
Copy link
Collaborator

Squashing and merging

@barendgehrels barendgehrels merged commit 91700dc into boostorg:develop Oct 9, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants